Skip to content

chore(security): fill the security-contact placeholders (owner-supplied values) - #33

Merged
hyperpolymath merged 1 commit into
mainfrom
chore/security-contact-fill
Jul 28, 2026
Merged

hyperpolymath merged 1 commit into
mainfrom
chore/security-contact-fill

Conversation

@hyperpolymath

Copy link
Copy Markdown
Owner

Fills all 13 {{PLACEHOLDER}} tokens in .github/SECURITY.md + .well-known/security.txt with the owner-ruled values (email security@metadatastician.art, key URL on metadatastician.art, fingerprint ABF3ECD932C2A04345EC9B3D2E725C6B831EA2B8, website). Should turn openssf-compliance green.

Three judgement calls, detailed in the commit message: template-instructions block deleted (it says to), gpg examples re-keyed on the fingerprint (the key's UID is jonathan@, so email-keyed lookups would fail), and the optional Hiring: line dropped (no /careers page — dead link).

Key publication itself rides on the estate-wellknown worker deploy (prepared, awaiting wrangler login).

…ed values)

Fills the 13 {{PLACEHOLDER}} tokens that kept openssf-compliance red, with
values ruled by the owner on 2026-07-27/28:

  SECURITY_EMAIL   security@metadatastician.art
  PGP_KEY_URL      https://metadatastician.art/.well-known/pgp-key.txt
  PGP_FINGERPRINT  ABF3ECD932C2A04345EC9B3D2E725C6B831EA2B8
                   (ed25519, created 2026-07-28, expires 2027-07-28,
                    verified from the owner's exported public key)
  WEBSITE          metadatastician.art

Judgement calls, recorded:
- The TEMPLATE INSTRUCTIONS comment block is deleted (its own text says to
  delete it before publishing, and it contained literal {{...}} tokens that
  would keep the gate red).
- The two gpg example commands were keyed on the SECURITY EMAIL; the key's
  UID is jonathan@metadatastician.art, so 'gpg --fingerprint security@...'
  and '--recipient security@...' would fail. Both now key on the FINGERPRINT,
  which works regardless of UID.
- security.txt's optional 'Hiring:' line is dropped rather than filled — no
  /careers page exists, and RFC 9116 marks the field optional. Shipping a
  dead link helps nobody.

The key itself is being published at the PGP_KEY_URL via the estate-wellknown
Cloudflare worker (which generates /.well-known/security.txt dynamically for
the estate domains); until that deploy lands the URL 404s — the repo files
are correct either way.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@gitar-bot

gitar-bot Bot commented Jul 28, 2026

Copy link
Copy Markdown
CI failed: CI failed because the newly added '.githooks/' directory at the repository root violates the root shape check allowlist.

Overview

1 change-related failure found across 1 analyzed log. The root shape validation check rejected the newly introduced .githooks/ directory in the repository root.

Failures

Root Shape Check Failure (confidence: high)

  • Type: build
  • Affected jobs: 90186660655
  • Related to change: yes
  • Root cause: The script scripts/check-root-shape.sh failed because the newly added directory .githooks/ is not present on the allowed root entries list.
  • Suggested fix: Add .githooks/ to the root allowlist file (e.g., .machine_readable/root-allow.txt) or move the files into an appropriate existing subdirectory.

Summary

  • Change-related failures: 1 failure due to an unallowed directory (.githooks/) added at the repository root.
  • Infrastructure/flaky failures: 0
  • Recommended action: Update the root shape allowlist to include .githooks/ or relocate the files.
Code Review ✅ Approved

Replaces placeholders in SECURITY.md and security.txt with owner-supplied security contact details to achieve OpenSSF compliance. No issues found.

Auto-approved and auto-merge armed: No blocking issues found.
Please see Auto-approve Docs for details on setting custom approval criteria. — merges when pipeline and required approvals pass.

Tip

Comment Gitar fix CI or enable auto-apply: gitar auto-apply:on

Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Showing less information.

Comment with these commands to change the behavior for this request:

Auto-apply Compact
gitar auto-apply:on         
gitar display:verbose         

Was this helpful? React with 👍 / 👎 | Gitar

@sonarqubecloud

Copy link
Copy Markdown

@gitar-bot

gitar-bot Bot commented Jul 28, 2026

Copy link
Copy Markdown

⚠️ Gitar auto-approved this PR but could not enable auto-merge: auto-merge is disabled for this repository — enable "Allow auto-merge" in the repository settings.

@gitar-bot gitar-bot Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gitar has auto-approved this PR and enabled auto-merge (configure)

@hyperpolymath
hyperpolymath merged commit 0af1a2f into main Jul 28, 2026
42 of 45 checks passed
@hyperpolymath
hyperpolymath deleted the chore/security-contact-fill branch July 28, 2026 05:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant